Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Roxygen #440

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Roxygen #440

wants to merge 10 commits into from

Conversation

pachadotdev
Copy link
Contributor

Hi @DavisVaughan

I am using cpp11 with great success in the lab.

In order to reduce clutter in my workflow, I added some code to be able to roxygenise directly in the cpp files rather than document the functions by separate.

This PR is kinda minimal, as it focused only on the roxygen feature.

Best,

@pachadotdev
Copy link
Contributor Author

@amoeba @hadley @klmr @xhochy hi! any chance this can be merged? i need to show some progress in the lab
I added other relevant PRs too

@hadley
Copy link
Member

hadley commented Jan 27, 2025

Hi @pachadotdev unfortunately we're all busy with other projects and don't have time to review at the moment.

@pachadotdev
Copy link
Contributor Author

Hi @pachadotdev unfortunately we're all busy with other projects and don't have time to review at the moment.

Noted!
I will send some extra PRs in the meanwhile. The response is sufficient to calm the waters at UofT.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants